Dependency Review

OpenSSF Scorecard

Scanned Files

• docs/requirements-dev.txt
• pyproject.toml

I added some Unit Tests (average coverage improvement: +33.04%). Check it out here to merge it!

No files changed in the PR

I also run pre commit on the whole dir for

I added some Unit Tests (average coverage improvement: +50.00%). Check it out here to merge it!

I'm just wondering, is there a reason not to use keyword arguments?
I believe you could have more understandable code, or at least not as many if elses as in the following picture.

The example would be:

def search_on_web(
    query: str,
    search_engine: str = "Google",
    max_results: int = 10,
    port: int = 8080,
    timeout: int = 10,
    proxy: str | dict = None,
    serper_api_key: str = None,
    **kwargs
) -> List[str]:
...
 google_search(
                        query,
                        num_results=max_results,
                        proxy=formatted_proxy,
                        **kwargs
                    )

You probably have a reason, you can just ignore this comment in any case. Thx again for the great package =)

This is the definition fo the search function from google_search library.

As you can see is not present the default region

I added some Unit Tests (average coverage improvement: +50.00%). Check it out here to merge it!

I see what you mean @VinciGit00 , however if your parameters are in the dictionary of parameters, there are going to be filled with the ** operator. Example:

def display_info(name, age):
    print(f"Name: {name}, Age: {age}")

person_info = {
    'name': 'Alice',
    'age': 30
}

display_info(**person_info)

@silgon take a look now

I added some Unit Tests (average coverage improvement: +50.00%). Check it out here to merge it!

Nice! 😎

Job has been stopped

No tests were needed for this PR, I think. If you were expecting tests, check out the documentation here to fix the issue

No tests were needed for this PR, I think. If you were expecting tests, check out the documentation here to fix the issue

@PeriniM I removed google search paragraph because it was not working anymore, probably due to new google policies

No tests were needed for this PR, I think. If you were expecting tests, check out the documentation here to fix the issue

🎉 This PR is included in version 1.36.1-beta.1 🎉

The release is available on:

• v1.36.1-beta.1
• GitHub release

Your semantic-release bot 📦🚀

No tests were needed for this PR, I think. If you were expecting tests, check out the documentation here to fix the issue

🎉 This PR is included in version 1.37.0 🎉

The release is available on:

• v1.37.0
• GitHub release

Your semantic-release bot 📦🚀